Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

doc/how to add evaluation #4

Merged
merged 3 commits into from
Mar 21, 2018
Merged

doc/how to add evaluation #4

merged 3 commits into from
Mar 21, 2018

Conversation

Superjomn
Copy link
Collaborator

No description provided.

@Superjomn Superjomn force-pushed the doc/how_to_add_evaluation branch from b47368d to e4fa70d Compare March 21, 2018 04:21

## Concepts

- `baseline`, one of the best KPI records in the history, the baseline should keep being updated.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be a running average or something similar? I'm afraid that the "best" one will be too good for the "normal" ones. But we can revisit this in the future.

doc/howto.md Outdated
## Concepts

- `baseline`, one of the best KPI records in the history, the baseline should keep being updated.
- `Factor`, an alias for KPI to make the code more readable.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

explain more about this?

doc/howto.md Outdated
from core import CostFactor, DurationFactor

train_cost_factor = CostFactor('train_cost', 0.15)
train_duration_factor = CostFactor('train_duration', 0.15)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be DurationFactor?

@Superjomn Superjomn force-pushed the doc/how_to_add_evaluation branch from e4fa70d to 9b3915f Compare March 21, 2018 05:50
@Superjomn Superjomn force-pushed the doc/how_to_add_evaluation branch from 9b3915f to a55c804 Compare March 21, 2018 05:52
@Superjomn Superjomn merged commit 06a1b07 into master Mar 21, 2018
@Superjomn Superjomn deleted the doc/how_to_add_evaluation branch March 21, 2018 06:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants